[스프링 화요일 1팀](이우식) 미션 제출합니다. by yiReal · Pull Request #21 · TEAM-ALOM/java-lotto

yiReal · 2024-03-16T09:47:59Z

Java 언어가 익숙치 않아 구글링과 GPT를 마구마구 참고했습니다...
가급적 MVC 형식에 맞춘 연습을 진행했습니다.

자바 문법이 어색하여 구글과 지피티를 열심히 참고하였습니다... MVC 형식에 최대한 적합하게끔 설게해보았습니당

자바가 친숙하지 않아 구글링과 GPT를 많이 참고하였습니다... 최대한 MVC형식에 맞게끔 작성해보았습니다.

EscapeSleep · 2024-03-18T08:15:27Z

src/main/java/lotto/domain/Lotto.java

+        }
+    }
+    private void validateRange(List<Integer> numbers){
+        for (int winNumber = 0;winNumber<numbers.size();winNumber++){


여기서 number.size 같은 것들은 상수로 사용하면 좋을 것 같습니다

넵 갑사합니다!

EscapeSleep · 2024-03-18T08:31:40Z

src/main/java/lotto/domain/LottoNumbers.java

+import java.util.List;
+import camp.nextstep.edu.missionutils.Randoms;
+
+public class LottoNumbers {


lottoNumbers이란 클래스 명은 lotto의 numbers 필드와 혼동할 수 있으므로 lottoUtils 같은 걸로 변경하면 좋아보입니다

EscapeSleep · 2024-03-18T08:46:38Z

src/main/java/lotto/domain/PlayerLottoAmount.java

+    public int calcLottoCount(){
+        return coin / lottoPrice;
+    }
+    private void validateCoin(int coin){


다른 에러가 아닌 같은 IllegalArgumentException 을 던지므로 한 로직에서 수행해도 될 것 같아요

EscapeSleep · 2024-03-18T08:48:30Z

src/main/java/lotto/domain/PlayerLottoAmount.java

+        validateNatural(coin);
+        validateDivisible(coin);
+    }
+    private static int validateNumber(String coin) throws IllegalArgumentException{


컨트롤러에서 이미 예외 핸들링을 하므로 예외 핸들링이 아닌 예외 던지기만 해도 될 것 같아요.

지구던지기는 안되나요?

EscapeSleep · 2024-03-19T00:08:31Z

src/main/java/lotto/domain/WinResult.java

+        this.lotto = lotto;
+        this.bonus = bonus;
+    }
+    public Ranking match(Lotto playername){


생성자의 파라미터 lotto의 이름을 로또가 아닌 winnigNumbers정도로 하면 좋을 것 같아요 winResult는 당첨번호들을 가진 객체이고 player 마다 로또를 하나씩 갖는 게 아닌 여러개를 가지므로 playerName을 lotto로 변경하면 좋을 것 같아요.

네이밍할 때 playername은 혼동을 줄 수 있으므로 카멜스타일로 playerName으로 바꾸면 좋을 것 같아요

카멜케이스를 잠시 망각했습니다 감사합니다 ㅎㅎ

EscapeSleep · 2024-03-19T00:09:49Z

src/main/java/lotto/view/ExceptionMessage.java

+    private static final String NOT_NUMBER_SIZE = "[ERROR] 당첨 번호는 6개 입력 가능합니다.";
+    private static final String NOT_NUMBER_OVERLAP = "[ERROR] 중복된 숫자를 입력하셨습니다.";
+    private static final String INPUT_TYPE_ERROR = "[ERROR] 숫자만 입력해 주세요.";
+    private static final int LOTTO_MIN_AMOUNT = 1000;


이런 상수들은 한 곳에 모아두는 편이 읽기 편할 것 같아요.

넵 정상수네요!

EscapeSleep · 2024-03-19T00:11:38Z

src/main/java/lotto/view/InputView.java

+        return winningNumberList;
+    }
+
+    private static int conventToInt(String inputNumber) {


convert가 오타가 난 것 같아요

헉 멋진 내가 이런 실수를! 감사합니다 ㅎㅎ

DaeunSon · 2024-03-19T06:09:22Z

src/main/java/lotto/controller/LottoController.java

try catch 구문에서 예외처리 할 때 Exception.printStackTrace()를 호출하셨는데 이대로 실행하면 에러 메세지가 구체적으로 뜨는건가요?(저는 잘 모르겠어서 질문 드려요..) 에러 원인이 무엇인지 구체적으로 뜨면 좋을 것 같아서 여쭤봅니다!!

이거 참고하시면 좋을것같아요! 코드가 길어지면 추적하기 유리할수록 좋을것같습니당

DaeunSon · 2024-03-19T06:25:24Z

src/main/java/lotto/domain/Ranking.java

+    public int getWinAmount() {
+        return winAmount;
+    }
+    private boolean counter(int matchCount){


counter 함수는 일치한 볼을 세는 함수니까 함수의 기능을 눈에 확 들어오는 이름으로 변경하면 더 좋을 것 같아요. 예를 들어 getMatchCounter / isMatchCounter 를 사용해도 좋을 것 같습니다

제가 적다가 손가락이 귀찮았나봅니다! 감사합니다 ㅎㅎ

DaeunSon · 2024-03-19T06:39:12Z

src/main/java/lotto/domain/LottoNumbers.java

+    public static List<Integer> setRandomNumbers(){
+        lottoNumberList = Randoms.pickUniqueNumbersInRange(minLottoNumber,maxLottoNumber,cntLottoNumber);
+        List<Integer> lottoTicketNumberList = new ArrayList<>(lottoNumberList);
+        Collections.sort(lottoTicketNumberList);


저는 리스트를 스트림으로 변환하고 sorted 기능을 사용했는데, 그냥 리스트에서 sort를 호출해서 사용하는게 더 직관적이고 쉬운 것 같아서 좋은 것 같습니다! 참고하겠습니다~

참고 하세요!

yiReal added 2 commits March 16, 2024 18:35

Java-lotto 과제 제출 - 이우식

6e7514d

자바 문법이 어색하여 구글과 지피티를 열심히 참고하였습니다... MVC 형식에 최대한 적합하게끔 설게해보았습니당

과제제출 - 이우식

608d462

자바가 친숙하지 않아 구글링과 GPT를 많이 참고하였습니다... 최대한 MVC형식에 맞게끔 작성해보았습니다.

yiReal changed the title ~~[스프링 화요일 1팀]이우식 미션 제출합니다.~~ [스프링 화요일 1팀](이우식) 미션 제출합니다. Mar 17, 2024

EscapeSleep reviewed Mar 19, 2024

View reviewed changes

DaeunSon reviewed Mar 19, 2024

View reviewed changes

Conversation

yiReal commented Mar 16, 2024

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants